Skip to content

fix: test do not accept multple line string on linux #756

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CSUwangj
Copy link

What:
using shell-quote to deal with multiple line string test case

Why and How:
current plugin accept multiple line string test case, but will lead to run time failure which said something like

balabala" is not a string

it's because arguments pass to leetcode-cli will be something like 123"\n"123, using shell-quote will resolve problem, for the "123"\n"123", it will turned into "'\"123\"\\n\"123\"'".

Beware that this solution only apply for Linux platform, I tested in my Ubuntu machine.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant